src/firebase/firebase.ts (1)

1-42: Firebase 관련 코드 완전 비활성화 및 의존성 정리 필요
현재 src/firebase/firebase.ts의 Firebase 초기화, 토큰 생성, 서비스워커 등록 코드가 모두 주석 처리되어 실제로 사용되지 않습니다. 그러나 package.json에는 여전히 "firebase": "^12.0.0" 의존성이 남아 있고, 환경 변수(VITE_FIREBASE_API_KEY 등) 설정도 제거되지 않았습니다. 의도된 중단이라면 관련 의존성과 설정을 함께 정리해주세요.

• package.json에서 "firebase" 의존성 제거
• .env 및 Vite 설정 파일에서 VITE_FIREBASE_*, VITE_PROJECT_ID 등 Firebase 관련 환경 변수 삭제
• src/firebase/firebase.ts 또는 전체 코드베이스에서 Firebase 참조 파일·주석 제거 또는 파일 삭제

src/hooks/alarm/useDeviceToken.ts (1)

1-40: useDeviceToken 훅이 완전히 비활성화되었습니다.

모든 구현 코드가 주석 처리되어 FCM 토큰 생성 및 서버 전송 기능이 비활성화되었습니다. 이는 디바이스 토큰 기반 푸시 알림 시스템의 완전한 중단을 의미합니다.

Firebase 기능 비활성화가 임시적인 조치라면, 나중에 쉽게 복구할 수 있도록 별도의 브랜치나 백업을 고려해보세요. 또는 기능을 완전히 제거할 계획이라면 관련 파일들과 의존성을 정리하는 것을 도와드릴 수 있습니다.

Configuration used: .coderabbit.yaml
Review profile: ASSERTIVE
Plan: Pro

src/pages/setting/DeleteReasonPage.tsx.tsx (3)

49-51: 선택한 탈퇴 사유를 전달하고, 미선택 시 ‘다음’ 버튼 비활성화가 필요합니다

선택 상태가 다음 화면으로 전달되지 않아 UX/기능 상 손실이 있습니다. 또한 미선택 상태에서도 이동이 가능해 오동작 우려가 있습니다. state로 넘기고, 미선택 시 비활성화를 권장합니다.

다음과 같이 수정 제안드립니다:

-                        <Link to="/deleteAccount/confirm" className="bg-primary-500 text-white px-20 py-3 rounding-16 font-body1">
-                            다음
-                        </Link>
+                        <Link
+                            to="/deleteAccount/confirm"
+                            state={{ reasons: selected }}
+                            className={`bg-primary-500 text-white px-20 py-3 rounded-16 font-body1 ${selected.length === 0 ? 'pointer-events-none opacity-50' : ''}`}
+                            aria-disabled={selected.length === 0}
+                        >
+                            다음
+                        </Link>

• 참고: Tailwind 클래스는 보통 rounded-…를 사용합니다. 기존 rounding-16은 오타로 보이며 rounded-16(또는 rounded-[16px])로 변경했습니다.

---

1-1: 파일 확장자(.tsx.tsx)로 인해 빌드/타입체크 누락 위험이 큽니다

현재 파일명이 ‘DeleteReasonPage.tsx.tsx’로 보입니다. 일반적으로 TSX 파일은 확장자가 .tsx 하나여야 합니다. 번들러/TS가 인식하지 못해 라우팅/빌드 실패로 이어질 수 있습니다. 즉시 파일명을 .tsx로 변경해 주세요.

프로젝트 루트에서 다음과 같이 리네임하는 것을 권장합니다:

# 예시: git으로 파일명 변경
git mv src/pages/setting/DeleteReasonPage.tsx.tsx src/pages/setting/DeleteReasonPage.tsx

---

5-5: 파일명과 경로 일관성 확인

• src/components/common/ReansonButton.tsx 파일이 실제로 존재합니다.
• 따라서 현재 import ReasonButton from '@/components/common/ReansonButton'; 는 올바른 경로입니다.
• 다만 파일명(ReansonButton.tsx)과 컴포넌트명(ReasonButton)이 불일치해 혼란을 줄 수 있습니다.

제안:

• 컴포넌트명에 맞춰 파일명도 ReasonButton.tsx 로 변경하세요.
• 기존 파일을 리네임한 뒤 모든 import 경로를 일괄 수정하시길 권장드립니다.

src/pages/setting/PaymentHistory.tsx (3)

70-72: 리스트 key는 index 대신 고유 식별자를 사용해 주세요

React key로 index를 쓰면 재정렬/삽입 시 의도치 않은 재사용이 발생할 수 있습니다. 주문번호가 고유값이라면 이를 key로 사용하는 편이 안전합니다.

다음과 같이 수정 제안드립니다:

-                            {dummyData.map((item, index) => (
-                                <PaymentRow key={index} data={item} />
+                            {dummyData.map((item) => (
+                                <PaymentRow key={item.orderNumber} data={item} />

---

90-90: ‘다음 페이지’ 아이콘에 동작을 부여해 주세요

현재 아이콘이 정적입니다. 클릭 시 다음 페이지로 넘어가도록 onClick을 부여하고, 마지막 페이지에서 비활성화/무시 처리를 권장합니다.

다음과 같이 수정 제안드립니다:

-                    <Forward className=" fill-current text-default-gray-500" />
+                    <button
+                        onClick={() => setCurrentPage((p) => Math.min(p + 1, 5))}
+                        aria-label="다음 페이지"
+                        className="disabled:opacity-50"
+                        disabled={currentPage === 5}
+                    >
+                        <Forward className="fill-current text-default-gray-500" />
+                    </button>

---

11-33: 더미 데이터 중 주문번호가 중복됩니다(키 고유성/표시 신뢰성 영향)

‘202219487’가 2회 등장합니다. 실제 연동 전이라도 더미 데이터의 식별자 중복은 key/테스트/UX에 혼선을 줄 수 있으니 고유하도록 정리해 주세요. 또한 실제 연동 시에는 타입(인터페이스) 명시를 권장합니다.

src/components/common/EditableInputBox.tsx (2)

91-98: 닉네임 readOnly일 때 ‘수정’ 버튼 노출/동작 차단 필요

현재 readOnly=true인 경우에도 (닉네임 모드에서) ‘수정’ 버튼이 노출되어 편집 상태로 진입할 수 있습니다. 읽기 전용을 확실히 보장하려면 readOnly일 때 버튼을 숨기거나 비활성화해 주세요.

예시:

• 버튼 노출 조건에 readOnly 배제
• onClick 보호 로직 추가

{isNickname && !isEditing && !readOnly && (
  <button
    /* type="button"도 함께 권장 */
    className="absolute right-3 top-1/2 -translate-y-1/2 font-body1 px-3 py-1 rounded-full bg-default-gray-400 text-default-gray-700"
    onClick={() => setIsEditing(true)}
  >
    수정
  </button>
)}

---

91-98: 버튼 type 명시 누락

폼 내에서의 오동작을 방지하려면 ‘수정’ 버튼에 type="button"을 명시해 주세요.

예시:

<button
  type="button"
  onClick={() => setIsEditing(true)}
  className="absolute right-3 top-1/2 -translate-y-1/2 font-body1 px-3 py-1 rounded-full bg-default-gray-400 text-default-gray-700"
>
  수정
</button>

src/types/notice/notice.ts (1)

11-18: 페이지네이션 메타의 기준(0/1-based) 주석을 명시해 주세요

currentPage, totalPages, currentSize의 의미(0/1 기반, 서버 규약)를 주석으로 고정하면 UI·API 간 불일치로 인한 버그를 줄일 수 있습니다. 현재 페이지 전송은 0-based로 보입니다(Notice.tsx에서 currentPage - 1). 타입 주석에 간략히 명시 부탁드립니다.

src/hooks/settingAlarm/useAlarms.ts (1)

4-4: 타입 임포트 네이밍 수정 완료 — 빌드 오류 해결됨

T 접두사 타입으로 정상화되어 과거 정적 분석 오류가 해소되었습니다. 현재 임포트/사용 모두 일관적입니다.

src/api/settingAlarm/alarm.ts (1)

7-8: withCredentials 중복 제거 반영 확인 — 간결하고 좋습니다

인스턴스 기본값을 신뢰하도록 정리되어 중복 옵션이 제거되었습니다. 유지보수성과 가독성이 개선됩니다.

Also applies to: 13-14

src/components/faq/FAQItem.tsx (1)

34-36: 아코디언 애니메이션의 max-height를 동적으로 측정하는 것을 권장합니다.

고정값 max-h-[1000px]는 긴 콘텐츠에서 끊기는 문제가 생깁니다. scrollHeight를 활용해 콘텐츠 높이를 측정해 주세요. (이전 코멘트와 동일 제안)

예시 구현:

// 컴포넌트 상단
const [contentHeight, setContentHeight] = useState(0);
const contentRef = useRef<HTMLDivElement>(null);

useEffect(() => {
  if (contentRef.current) {
    setContentHeight(contentRef.current.scrollHeight);
  }
}, [item.content, isOpen]);

// 섹션 요소
<section
  id={contentId}
  aria-labelledby={`faq-toggle-${item.faqId}`}
  aria-hidden={!isOpen}
  className="ml-0 sm:ml-9 overflow-hidden transition-[max-height,opacity] duration-200 ease-in-out"
  style={{ maxHeight: isOpen ? contentHeight : 0, opacity: isOpen ? 1 : 0 }}
>
  <div ref={contentRef} className="px-4 py-3 rounded-md leading-relaxed whitespace-pre-line text-default-gray-700 font-body2 bg-white/50">
    {item.content}
  </div>
</section>

src/pages/question/Question.tsx (1)

82-82: 빈 onSearchClick 콜백 정리 또는 의미 있는 동작으로 대체

불필요한 빈 함수는 제거하거나, 페이지/상태 초기화 등 실제 동작을 연결해 주세요. (이전 코멘트와 동일 맥락)

예시:

-                onSearchClick={() => {}}
+                onSearchClick={() => {
+                    setCurrentPage(1);
+                    setOpenedIndex(null);
+                }}

src/components/common/PasswordEdit.tsx (1)

79-86: 취소 버튼에 type 명시 필요

취소 버튼에 type="button"을 명시해 폼 내에서의 의도치 않은 submit을 방지해 주세요.

다음과 같이 수정:

-                    <button
+                    <button
+                        type="button"
                         onClick={handleCancel}
                         className="px-4 py-1.5 rounding-16 bg-default-gray-400 text-default-gray-700 font-body1 disabled:opacity-60"
                         disabled={isPending}
                     >
                         취소
                     </button>

src/pages/notice/Notice.tsx (2)

30-35: 훅 규칙 위반 수정 확인 — 조건부 호출 제거 Good

두 쿼리를 항상 호출하고 enabled로 토글하는 패턴으로 수정되어 안전합니다.

---

60-71: 카테고리 버튼 type 명시 반영 확인

type="button"이 추가되어 폼 제출 부작용이 방지되었습니다.

src/hooks/notice/useNotice.ts (2)

27-34: params 안정화(useMemo) 처리 깔끔합니다

이전 피드백대로 새로운 객체를 구성해 참조 안정성을 보장한 점 좋습니다.

---

45-45: 공지 상세 enabled 조건을 정수로 한정

음수/실수 방지를 위해 Number.isFinite 대신 정수 체크를 권장합니다.

-            enabled: Number.isFinite(id) && id > 0,
+            enabled: Number.isInteger(id) && id > 0,

Configuration used: .coderabbit.yaml
Review profile: ASSERTIVE
Plan: Pro

src/pages/notice/NoticeDetail.tsx (2)

9-11: 유효하지 않은 ID 처리 순서 개선 필요

ID 유효성 검사를 훅 호출 이전에 수행하면 불필요한 API 호출을 방지할 수 있습니다.

---

16-16: 주석이 실제 흐름과 반대로 적혀 있습니다

지금은 훅 호출 이후에 유효성 검사를 통해 렌더링을 가드합니다. 혼동을 피하려면 주석을 "Id 유효성 검사 -> 렌더링 가드" 등으로 정정해 주세요.

src/api/auth/account.ts (3)

8-9: withCredentials 중복 설정 제거 권장

axiosInstance에서 이미 withCredentials: true를 기본값으로 설정하고 있으므로, 개별 요청 옵션에서 중복으로 명시할 필요가 없습니다.

---

13-16: 닉네임 변경 API 401 에러 처리 필요

PR 설명에서 닉네임 변경 API도 401 에러를 반환한다고 명시되어 있습니다. 비밀번호 변경과 동일하게 인증 에러 처리가 필요합니다.

---

7-10: changePassword 반환 타입과 훅 제네릭 불일치

changePassword가 Promise<void>를 반환하지만, 훅에서는 TChangePasswordResponse로 사용 중인 것으로 보입니다. 훅의 제네릭을 void로 맞춰야 타입 안전성을 확보할 수 있습니다.

src/hooks/notice/useNotice.ts (3)

53-62: TypeScript 제네릭 타입 오류 수정 필요

동일한 TypeScript 제네릭 타입 오류가 있습니다.

-    const useSearchNotices = (params: TSearchParams, options?: Parameters<typeof useCoreQuery<TFetchNoticesResponse>>[2]) => {
+    const useSearchNotices = (params: TSearchParams, options?: Parameters<typeof useCoreQuery>[2]) => {

---

28-43: TypeScript 제네릭 타입 오류 수정 필요

Parameters<typeof useCoreQuery<TFetchNoticesResponse>>[2] 형태는 TypeScript에서 허용되지 않습니다. typeof에는 제네릭을 적용할 수 없으므로 수정이 필요합니다.

-    const useGetNotices = (params: TListParams, options?: Parameters<typeof useCoreQuery<TFetchNoticesResponse>>[2]) => {
+    const useGetNotices = (params: TListParams, options?: Parameters<typeof useCoreQuery>[2]) => {

---

46-50: TypeScript 제네릭 타입 오류 수정 필요

동일한 TypeScript 제네릭 타입 오류가 있습니다.

-    const useGetNoticeDetail = (id: number, options?: Parameters<typeof useCoreQuery<TFetchNoticeDetailResponse>>[2]) =>
+    const useGetNoticeDetail = (id: number, options?: Parameters<typeof useCoreQuery>[2]) =>

src/hooks/auth/useAccount.ts (1)

18-40: TypeScript 타입 오류로 인한 빌드 실패가 해결되었습니다

이전 커밋에서 지적된 useCoreMutation 타입 불일치 문제가 src/hooks/customQuery.ts의 제네릭 타입 개선으로 해결되어, 이제 적절한 타입 매개변수가 자동으로 전달됩니다.

Configuration used: .coderabbit.yaml
Review profile: ASSERTIVE
Plan: Pro

src/components/settingTab/InfoSetting.tsx (3)

56-61: 닉네임 제출 로직의 trim/중복 방지/중복요청 방어가 잘 반영됨

공백 입력 차단, 동일 값 변경 방지, pending 상태 방지 모두 적절합니다. 이전 제안사항이 반영되었습니다.

---

91-99: 취향 데이터 초기화 버튼: 타입/핸들러 추가 반영 — 좋습니다

type="button" 지정과 onClick 핸들러 연결이 적절합니다. 이전 피드백이 반영되었습니다.

---

78-85: 이메일 입력: pointer-events-none 제거 권장 (접근성 개선)

readOnly만으로도 입력 방지 목적을 달성합니다. pointer-events-none은 텍스트 선택/포커스를 막아 접근성을 해칩니다.

다음과 같이 수정해 주세요.

             <EditableInputBox
                 label="이메일"
-                value={infoLoading ? '불러오는 중' : infoError ? '' : email}
-                readOnly
-                onChange={() => {}}
-                className="pointer-events-none"
+                value={infoLoading ? '불러오는 중' : infoError ? '' : email}
+                readOnly
+                onChange={() => {}}
                 placeholder="이메일"
             />

src/api/auth/account.ts (2)

7-10: changePassword가 Promise를 반환 — 훅 제네릭과 정렬 필요

useAccount의 useChangePassword 제네릭이 응답 타입으로 TChangePasswordResponse를 기대하고 있습니다. 현재 구현과 불일치합니다. 훅 제네릭의 첫 번째 타입을 void로 정렬해 주세요.

다음과 같이 수정(파일: src/hooks/auth/useAccount.ts):

- function useChangePassword(options?: TChangePasswordMutationOptions): TChangePasswordMutationResult {
-   return useCoreMutation<TChangePasswordResponse, TChangePasswordPayload>(changePassword, options);
- }
+ function useChangePassword(options?: TChangePasswordMutationOptions): TChangePasswordMutationResult {
+   return useCoreMutation<void, TChangePasswordPayload>(changePassword, options);
+ }

---

4-5: withCredentials 기본 설정 활용 — 개별 요청 중복 설정 없음 (좋습니다)

axiosInstance가 withCredentials: true를 기본으로 가지므로 개별 요청에서 중복 지정이 없고, 현재 구현이 간결합니다.

src/hooks/notice/useNotice.ts (4)

28-28: TypeScript 컴파일 에러 해결이 필요합니다.

typeof useCoreQuery<TFetchNoticesResponse>에서 제네릭 타입을 사용할 수 없어 빌드가 실패합니다.

다음과 같이 수정해 주세요:

-    const useGetNotices = (params: TListParams, options?: Parameters<typeof useCoreQuery<TFetchNoticesResponse>>[2]) => {
+    const useGetNotices = (params: TListParams, options?: Parameters<typeof useCoreQuery>[2]) => {

---

46-46: TypeScript 컴파일 에러 해결이 필요합니다.

동일한 제네릭 타입 이슈가 있습니다.

-    const useGetNoticeDetail = (id: number, options?: Parameters<typeof useCoreQuery<TFetchNoticeDetailResponse>>[2]) =>
+    const useGetNoticeDetail = (id: number, options?: Parameters<typeof useCoreQuery>[2]) =>

---

53-53: TypeScript 컴파일 에러 해결이 필요합니다.

동일한 제네릭 타입 이슈입니다.

-    const useSearchNotices = (params: TSearchParams, options?: Parameters<typeof useCoreQuery<TFetchNoticesResponse>>[2]) => {
+    const useSearchNotices = (params: TSearchParams, options?: Parameters<typeof useCoreQuery>[2]) => {

---

64-68: 훅 팩토리 패턴의 장단점 고려해 보세요.

현재 구조는 동작상 문제없지만, 개별 훅을 최상위로 내보내는 방식도 고려해 볼 만합니다.

트리 셰이킹과 IDE 지원 개선을 위해 다음과 같은 구조도 검토해 보시기 바랍니다:

export const useGetNotices = (params: TListParams, options?: Parameters<typeof useCoreQuery>[2]) => {
    // 현재 로직
};

export const useGetNoticeDetail = (id: number, options?: Parameters<typeof useCoreQuery>[2]) => {
    // 현재 로직  
};

export const useSearchNotices = (params: TSearchParams, options?: Parameters<typeof useCoreQuery>[2]) => {
    // 현재 로직
};

Configuration used: .coderabbit.yaml
Review profile: ASSERTIVE
Plan: Pro

src/components/common/EditableInputBox.tsx (1)

7-7: type prop의 타입 정의가 제한적입니다

현재 type prop이 'text'만 허용하도록 정의되어 있는데, 이는 확장성이 부족합니다. password, email, tel 등 다른 HTML input 타입도 지원할 수 있도록 개선이 필요합니다.

-    type?: 'text';
+    type?: React.HTMLInputTypeAttribute;

또는 필요한 타입만 제한적으로 허용하려면:

-    type?: 'text';
+    type?: 'text' | 'password' | 'email' | 'tel' | 'number';

src/routes/routes.tsx (1)

32-41: ProtectedRoute의 인증 로직이 하드코딩되어 있습니다

현재 isLoggedIn이 항상 true로 설정되어 있어 실제 인증 검증이 이루어지지 않습니다. TODO 주석이 있지만, 보안상 중요한 부분이므로 실제 인증 상태를 확인하는 로직으로 교체가 시급합니다.

실제 인증 상태를 확인하는 커스텀 훅을 사용하도록 수정이 필요합니다:

 function ProtectedRoute({ children }: PropsWithChildren) {
-    //추후 실제 로그인 여부로 대체 필요
-    const isLoggedIn = true;
+    const { isAuthenticated } = useAuth(); // 실제 인증 훅 사용
 
-    if (!isLoggedIn) {
+    if (!isAuthenticated) {
         return <Navigate to="/" replace />;
     }

인증 훅 구현이 필요하시면 도와드릴까요?

src/utils/date.ts (1)

2-4: 유효하지 않은 날짜 문자열 처리 반영, 좋습니다

Falsy 입력 및 Invalid Date 가드가 추가되어 안정성이 개선되었습니다. 이전 코멘트에서 지적되던 부분이 해결되었습니다.

src/components/common/EditableInputBox.tsx (1)

17-19: readOnly prop이 이제 제대로 사용되고 있습니다!

이전 리뷰에서 지적된 readOnly prop 미사용 문제가 해결되었습니다. prop이 적절히 구조 분해 할당되고 실제로 사용되고 있습니다.

src/routes/routes.tsx (1)

98-98: 라우트 경로가 상대 경로로 올바르게 정의되었습니다

이전 리뷰에서 지적된 문제가 해결되었습니다. notice/:noticeId로 상대 경로를 사용하여 부모 라우트와 올바르게 결합됩니다.

src/components/settingTab/MembershipSetting.tsx (3)

37-47: 버튼 클릭 핸들러가 타입 기반으로 개선되었습니다

이전 리뷰에서 제안한 대로 레이블 문자열 의존성을 제거하고 타입 기반 버튼 모델로 리팩토링이 잘 완료되었습니다. disabled 속성과 state를 활용한 분기 처리가 깔끔합니다.

---

33-33: openMembership state가 실제로 사용되지 않을 수 있습니다

이전 리뷰에서 지적된 대로 PaymentHistory 컴포넌트에서 openMembership state를 실제로 읽지 않는다면 불필요한 state 전달입니다.

#!/bin/bash
# PaymentHistory에서 openMembership state 사용 여부 확인
echo "=== PaymentHistory.tsx에서 location.state 사용 검색 ==="
rg -A 5 "useLocation|location\.state" src/pages/setting/PaymentHistory.tsx || echo "location.state 사용 없음"

echo -e "\n=== openMembership 전체 사용처 검색 ==="
rg "openMembership" --type tsx --type ts

---

39-40: alert 대신 Toast 사용을 고려해보세요

이전 리뷰에서도 지적되었듯이 alert는 사용자 경험을 방해합니다. Toast나 모달 같은 비차단형 UI 컴포넌트 사용을 권장합니다.

-            alert('해당 기능은 추후 제공 예정입니다.');
+            showToast('해당 기능은 추후 제공 예정입니다.', { type: 'info' });

src/pages/setting/DeleteConfirmPage.tsx (3)

43-46: 에러 타입을 any 대신 unknown으로 변경하세요

타입 안전성을 위해 any 대신 unknown을 사용하고 적절한 타입 가드를 추가하는 것이 좋습니다.

-        onError: (error) => {
-            const msg = error?.response?.data?.message || '회원 탈퇴에 실패했습니다.';
+        onError: (error: unknown) => {
+            const msg = (error as any)?.response?.data?.message || '회원 탈퇴에 실패했습니다.';
             alert(msg);
         },

더 나은 방법은 AxiosError 타입 가드를 사용하는 것입니다:

+import { AxiosError } from 'axios';

         onError: (error: unknown) => {
-            const msg = error?.response?.data?.message || '회원 탈퇴에 실패했습니다.';
+            let msg = '회원 탈퇴에 실패했습니다.';
+            if (error instanceof AxiosError && error.response?.data?.message) {
+                msg = error.response.data.message;
+            }
             alert(msg);
         },

---

51-55: 사용자 경험 개선을 위해 커스텀 모달 사용을 고려하세요

브라우저 기본 confirm 대신 앱의 디자인과 일치하는 커스텀 모달을 사용하면 더 나은 사용자 경험을 제공할 수 있습니다.

---

120-121: Tailwind CSS 표준 클래스를 사용하세요

rounding-32는 Tailwind CSS 표준 클래스가 아닙니다. rounded-full 또는 rounded-[32px]를 사용해야 합니다.

-            className={`px-8 py-3 rounding-32 font-body1 text-white
+            className={`px-8 py-3 rounded-full font-body1 text-white
                 ${allAgreed && !isPending ? 'bg-primary-500 hover:bg-primary-600' : 'bg-default-gray-400 cursor-not-allowed'}`}

src/components/settingTab/InfoSetting.tsx (3)

98-98: pointer-events-none 클래스를 제거하세요

readOnly 속성만으로도 입력을 방지할 수 있으며, pointer-events-none은 텍스트 선택과 접근성을 해칠 수 있습니다.

-                className="pointer-events-none"

---

41-42: 로그아웃 시 localStorage ‘nickname’ 제거 로직 추가 필요

현재 useLogout 훅을 호출하고 있지만, 로그아웃 시 localStorage에 남아 있는 nickname을 제거하는 코드가 없습니다. 아래 위치에 localStorage.removeItem('nickname')를 추가해주세요.

• 위치: src/hooks/auth/useAuth.ts
  • useLogout = useCoreMutation(logout) 정의부

예시(diff):

--- a/src/hooks/auth/useAuth.ts
+++ b/src/hooks/auth/useAuth.ts
@@ -10,7 +10,13 @@ export function useAuth() {
-  const useLogout = useCoreMutation(logout);
+  const useLogout = useCoreMutation(logout, {
+    onSuccess: () => {
+      // 로그아웃 후 닉네임 제거
+      localStorage.removeItem('nickname');
+    },
+  });

   return {
     useLogout,

---

29-33: 빈 문자열도 유효한 값으로 처리하도록 조건문을 수정하세요

현재 apiNickname이 빈 문자열일 때 상태가 업데이트되지 않습니다. 서버에서 빈 문자열을 반환할 수 있다면 이를 정상 값으로 처리해야 합니다.

     useEffect(() => {
-        if (apiNickname) {
+        if (apiNickname !== undefined && apiNickname !== null) {
             setNickname(apiNickname);
             setInitialNickname(apiNickname);
         }
     }, [apiNickname]);

또는 기본값 할당을 제거하는 방법도 있습니다:

-    const apiNickname = memberData?.result?.username ?? '';
+    const apiNickname = memberData?.result?.username;

Configuration used: .coderabbit.yaml
Review profile: ASSERTIVE
Plan: Pro

src/types/notice/notice.ts (1)

11-18: 목록 응답 타입 구조 OK + 재사용성 고려 제안

search 응답도 동일 구조를 반환한다면 현재 제네릭 래핑은 합리적입니다. 다만 Notice 목록 페이지네이션 페이로드를 별도 타입으로 분리해 재사용성을 높일 수 있습니다.

아래처럼 서브 타입을 분리하면 여러 API(search, list)가 동일한 result 구조를 공유할 때 유지보수가 편해집니다.

-export type TFetchNoticesResponse = TCommonResponse<{
-    noticeList: TNoticeItem[];
-    totalPages: number;
-    currentPage: number;
-    currentSize: number;
-    hasNextPage: boolean;
-}>;
+type TNoticeListPage = {
+  noticeList: TNoticeItem[];
+  totalPages: number;
+  currentPage: number;
+  currentSize: number;
+  hasNextPage: boolean;
+};
+export type TFetchNoticesResponse = TCommonResponse<TNoticeListPage>;

src/components/common/EditableInputBox.tsx (1)

155-160: 취소/완료 버튼에 type="button" 추가 필요

폼 내부에서 의도하지 않은 제출을 방지하기 위해 명시적인 type 속성이 필요합니다.

                 <div className="flex justify-end gap-2 mt-3">
-                    <button onClick={handleCancel} className="font-body1 px-4 py-1.5 rounded-full bg-default-gray-400 text-default-gray-700">
+                    <button type="button" onClick={handleCancel} className="font-body1 px-4 py-1.5 rounded-full bg-default-gray-400 text-default-gray-700">
                         취소
                     </button>
-                    <button onClick={handleSubmit} className="font-body1 px-4 py-1.5 rounded-full bg-default-gray-400 text-default-gray-700">
+                    <button type="button" onClick={handleSubmit} className="font-body1 px-4 py-1.5 rounded-full bg-default-gray-400 text-default-gray-700">
                         완료
                     </button>
                 </div>

src/api/notice/notice.ts (1)

24-37: search 파라미터 타입 분리로 일관성/재사용성 개선

인라인 타입 대신 별도 타입으로 분리해 재사용성과 가독성을 높이는 것이 좋습니다. 기존 리뷰에서도 동일 제안이 있었습니다.

아래처럼 타입을 분리해 주세요.

-export const searchNotices = async (params: { keyword: string; page: number; size: number; category?: 'SERVICE' | 'SYSTEM' }) => {
+export type TSearchNoticesParams = {
+  keyword: string;
+  page: number;
+  size: number;
+  category?: 'SERVICE' | 'SYSTEM';
+};
+export const searchNotices = async (params: TSearchNoticesParams) => {
     const { keyword, page, size, category } = params;

src/components/settingTab/InfoSetting.tsx (4)

23-33: apiNickname 동기화 조건 수정 필요(빈 문자열 처리 누락)

빈 문자열('')일 경우 useEffect가 동작하지 않아 상태가 갱신되지 않습니다. 기본값 할당을 제거하고 undefined/null 여부만 판단하도록 변경하세요.

아래처럼 수정 제안드립니다.

-    const apiNickname = memberData?.result?.username ?? '';
+    const apiNickname = memberData?.result?.username;

     useEffect(() => {
-        if (apiNickname) {
+        // apiNickname이 빈 문자열이어도 상태에 반영
+        if (apiNickname !== undefined && apiNickname !== null) {
             setNickname(apiNickname);
             setInitialNickname(apiNickname);
         }
     }, [apiNickname]);

---

95-99: 읽기 전용 이메일 입력에 pointer-events-none 제거 권장

readOnly만으로 입력 방지가 충분합니다. pointer-events-none은 텍스트 복사/포커스를 막아 접근성을 떨어뜨립니다.

아래처럼 className을 제거해 주세요.

             <EditableInputBox
                 label="이메일"
-                value={infoLoading ? '불러오는 중' : infoError ? '' : email}
-                readOnly
-                onChange={() => {}}
-                className="pointer-events-none"
+                value={infoLoading ? '불러오는 중' : infoError ? '' : email}
+                readOnly
+                onChange={() => {}}
                 placeholder="이메일"
             />

---

111-111: Tailwind 클래스 오타: rounding-32 → rounded-[32px] (또는 rounded-full)

Tailwind 표준 유틸이 아닙니다. 의도대로 32px 라운딩을 유지하려면 arbitrary value를 사용하세요.

-                    className={`bg-primary-500 px-6 py-2 rounding-32 text-white font-body1 ${resetPending ? 'opacity-60 cursor-not-allowed' : ''}`}
+                    className={`bg-primary-500 px-6 py-2 rounded-[32px] text-white font-body1 ${resetPending ? 'opacity-60 cursor-not-allowed' : ''}`}

---

35-48: 로그아웃 시 로컬스토리지 닉네임 제거 필요

현재 InfoSetting.tsx에서 닉네임을 localStorage.setItem('nickname', …)으로만 저장하고, 로그아웃 시 해당 키를 지우는 로직이 없습니다. 로그아웃 처리(예: 로그아웃 API 호출 후 성공 콜백) 부분에 반드시 localStorage.removeItem('nickname')을 추가해 주세요.

수정 위치 예시

• src/components/settingTab/InfoSetting.tsx
• 로그아웃 처리 컴포넌트/훅 (예: useAuth.ts에서 useLogout을 사용하는 곳)

예시 diff

// 로그아웃 성공 이후
onSuccess: () => {
+   localStorage.removeItem('nickname');
    // …기존 로그아웃 후 처리 로직
}

src/hooks/auth/useAccount.ts (1)

19-23: QUERY_KEYS는 모듈 스코프로 이동/공유하세요

훅 호출 시마다 객체가 재생성됩니다. 모듈 상단으로 이동하고(또는 공용 constants 폴더로 분리) 내외부에서 동일 상수를 사용하도록 해 주세요. InfoSetting의 invalidateQueries에서도 재사용 가능합니다.

아래처럼 이동/정리 제안드립니다.

+// 파일 상단(모듈 스코프)으로 이동
+const QUERY_KEYS = {
+    memberInfo: ['memberInfo'] as const,
+    memberGrade: ['memberGrade'] as const,
+} as const;

 export function useAccount() {
-    const QUERY_KEYS = {
-        memberInfo: ['memberInfo'] as const,
-        memberGrade: ['memberGrade'] as const,
-    } as const;

또는 src/constants/queryKeys.ts로 분리 후 import하여 사용하세요.

Configuration used: .coderabbit.yaml
Review profile: ASSERTIVE
Plan: Pro

src/hooks/alarm/useDeviceToken.ts (1)

1-40: 주석된 useDeviceToken 훅 정리 및 no-op 버전으로 교체

현재 src/hooks/alarm/useDeviceToken.ts 파일 전체가 주석 처리되어 있고, 실제 코드베이스에서는 HomePage.tsx에서만 호출문이 주석으로 남아 있습니다. 유지보수성과 린터 이슈를 해결하기 위해 아래와 같이 정리해 주세요.

• 확인된 사용처
  • src/pages/home/HomePage.tsx
    • 16행: // useDeviceToken(); 호출문만 존재(주석 처리)

제안 수정안:

-// src/hooks/alarm/useDeviceToken.ts
-// import { useEffect } from 'react';
-// import { isSupported } from 'firebase/messaging';
-// import { postDeviceToken } from '@/api/alarm/alarm'; 
-// import { generateToken, registerServiceWorker } from '@/firebase/firebase';
-//
-// export const useDeviceToken = () => {
-//   useEffect(() => {
-//     /* FCM 토큰 등록 로직 */
-//   }, []);
-// };
+// src/hooks/alarm/useDeviceToken.ts
+// 임시 비활성화: FCM 디바이스 토큰 등록 API가 401을 반환하여 로직 보류 (#90)
+export const useDeviceToken = (): void => {
+  // TODO(#90): FCM 토큰 등록 로직 재활성화
+};

이제 불필요한 주석 코드를 제거하고, 동일 시그니처의 no-op 훅을 유지해 사용처가 깨지지 않도록 해주세요.

src/components/common/EditableInputBox.tsx (1)

67-71: 검색 인풋에서 Enter 입력 시 기본 제출 방지 필요

폼 내부 사용 시 의도치 않은 submit을 유발할 수 있습니다. e.preventDefault()를 추가해 주세요.

     const handleKeyDown = (e: React.KeyboardEvent<HTMLInputElement>) => {
         if (e.key === 'Enter' && isSearch) {
-            onSearchClick?.();
+            e.preventDefault();
+            onSearchClick?.();
         }
     };

src/pages/setting/DeleteReasonPage.tsx.tsx (2)

5-6: 컴포넌트 파일명 오타 수정 필요

현재 src/components/common/ReansonButton.tsx 파일명이 오타로 인해 잘못 지정되어 있습니다. 아래 항목을 반영해 파일명과 import 구문을 일치시켜 주세요.

• 파일명 변경
  • src/components/common/ReansonButton.tsx → src/components/common/ReasonButton.tsx
• import 구문 수정 (DeleteReasonPage.tsx.tsx 5행)

  - import ReasonButton from '@/components/common/ReansonButton';
  + import ReasonButton from '@/components/common/ReasonButton';

---

1-1: 파일명 확장자 중복(.tsx.tsx) 수정 및 import 경로 동기화 필요

중복된 .tsx.tsx 확장자로 인해 에디터나 번들러가 파일을 인식하지 못할 수 있습니다. 아래 위치를 수정해 주세요:

• 파일 리네임

• 변경 전: src/pages/setting/DeleteReasonPage.tsx.tsx
• 변경 후: src/pages/setting/DeleteReasonPage.tsx

• import 경로 수정 (src/routes/routes.tsx 29행)

- import DeleteReasonPage from '@/pages/setting/DeleteReasonPage.tsx';
+ import DeleteReasonPage from '@/pages/setting/DeleteReasonPage';

이후 관련 경로들이 정상 동작하는지 빌드 및 에디터에서 확인 부탁드립니다.

src/pages/home/HomePage.tsx (1)

16-16: 주석 처리된 호출은 맥락을 남기거나 제거해 주세요

일시 비활성화라면 이유(401/이슈 번호)를 명시한 TODO로 남겨 주세요. 불필요하면 삭제가 깔끔합니다.

가능한 수정안:

-    // useDeviceToken();
+    // TODO(#90): 알림(FCM) 디바이스 토큰 전송 API가 401을 반환하여 임시 비활성화. 이슈 해결 후 재활성화.
+    // useDeviceToken();

src/api/notice/notice.keys.ts (1)

5-12: 검색/목록 키에 객체를 직접 넣으면 캐시 키가 매 호출마다 달라집니다 — 원시 파라미터로 분리하세요

React Query 키에 객체 참조를 포함하면 렌더마다 새로운 레퍼런스가 생겨 캐시 적중률이 떨어지고 불필요한 refetch를 유발합니다. keyword/page/size/category를 개별 인자로 받아 키를 구성하는 방식이 안전합니다.

아래와 같이 수정 제안드립니다.

 export const noticeKeys = {
     root: ['notice'] as const, // 루트 키

     // 목록 조회 키
-    list: (p: { category: 'SERVICE' | 'SYSTEM'; page: number; size: number }) => [...noticeKeys.root, 'list', p] as const,
+    list: (category: 'SERVICE' | 'SYSTEM', page: number, size: number) =>
+        [...noticeKeys.root, 'list', category, page, size] as const,

     // 상세 조회 키
     detail: (id: number) => [...noticeKeys.root, 'detail', id] as const,

     // 검색 조회 키
-    search: (p: { keyword: string; page: number; size: number; category?: 'SERVICE' | 'SYSTEM' }) => [...noticeKeys.root, 'search', p] as const,
+    search: (keyword: string, page: number, size: number, category?: 'SERVICE' | 'SYSTEM') =>
+        [...noticeKeys.root, 'search', keyword, page, size, ...(category ? [category] : [])] as const,
 };

호출부도 객체 대신 개별 인자로 넘기도록 함께 정리해 주세요.

src/components/faq/FAQItem.tsx (1)

19-37: role="region" 제거 및 버튼-패널 연계 라벨링 추가 필요

Biome가 지적하듯 section에 role="region"은 불필요하며, 이름 없는 landmark 이슈를 만듭니다. 버튼에 id를 부여하고 section에 aria-labelledby로 연결해 주세요.

-            <button type="button" onClick={onToggle} className="flex w-full items-center gap-4 text-left" aria-expanded={isOpen} aria-controls={contentId}>
+            <button
+                id={`faq-toggle-${item.faqId}`}
+                type="button"
+                onClick={onToggle}
+                className="flex w-full items-center gap-4 text-left"
+                aria-expanded={isOpen}
+                aria-controls={contentId}
+            >
...
-            <section
-                id={contentId}
-                role="region"
-                aria-hidden={!isOpen}
+            <section
+                id={contentId}
+                aria-labelledby={`faq-toggle-${item.faqId}`}
+                aria-hidden={!isOpen}
                 className={`ml-0 sm:ml-9 overflow-hidden transition-[max-height,opacity] duration-200 ease-in-out ${
                     isOpen ? 'max-h-[1000px] opacity-100 mt-4' : 'max-h-0 opacity-0'
                 }`}
             >

src/components/common/EditableInputBox.tsx (1)

149-155: 취소/완료 버튼에도 type="button"을 명시하세요

기본 type=submit로 인해 폼 내에서 의도치 않은 제출이 발생할 수 있습니다.

-                    <button onClick={handleCancel} className="font-body1 px-4 py-1.5 rounded-full bg-default-gray-400 text-default-gray-700">
+                    <button type="button" onClick={handleCancel} className="font-body1 px-4 py-1.5 rounded-full bg-default-gray-400 text-default-gray-700">
                         취소
                     </button>
-                    <button onClick={handleSubmit} className="font-body1 px-4 py-1.5 rounded-full bg-default-gray-400 text-default-gray-700">
+                    <button type="button" onClick={handleSubmit} className="font-body1 px-4 py-1.5 rounded-full bg-default-gray-400 text-default-gray-700">
                         완료
                     </button>

src/pages/setting/DeleteReasonPage.tsx.tsx (1)

32-34: 뒤로가기: 히스토리 우선, 홈 폴백으로 개선

딥링크/다단계 진입 시 컨텍스트 유지를 위해 history back을 우선하고, 없을 때만 홈으로 이동하세요. 접근성 라벨과 버튼 type도 함께 명시합니다.

-                        <button onClick={() => navigate('/home')}>
+                        <button
+                            type="button"
+                            aria-label="뒤로가기"
+                            onClick={() => (window.history.length > 1 ? navigate(-1) : navigate('/home'))}
+                        >

src/routes/routes.tsx (1)

98-98: 라우트 경로 불일치 문제

notice/:noticeId 경로가 상대 경로로 올바르게 정의되어 있습니다. 이전 리뷰에서 지적된 절대 경로 문제가 해결되었습니다.

src/types/notice/notice.ts (1)

39-39: 파일 끝에 개행 문자를 추가하세요

Prettier 규칙에 따라 파일 끝에 개행 문자가 필요합니다.

-export type TFetchNoticeDetailResponse = TCommonResponse<TNoticeDetail | null>;
+export type TFetchNoticeDetailResponse = TCommonResponse<TNoticeDetail | null>;
+

src/api/settingAlarm/alarm.ts (1)

1-15: React Query 취소 신호(AbortSignal) 전달을 위해 AxiosRequestConfig 지원을 추가해 주세요

쿼리 취소/중복 방지를 위해 API 함수가 signal을 전달받을 수 있어야 합니다. 선택적 config를 받아 axios로 넘기도록 확장하는 것을 권장합니다.

아래와 같이 수정 제안 드립니다:

 import type { TAlarmSettings, TGetAlarmSettingsResp, TPatchAlarmSettingsResp } from '@/types/settingAlarm/alarm';
 
 import { axiosInstance } from '@/api/axiosInstance';
+import type { AxiosRequestConfig } from 'axios';
 
 // 조회
-export async function getAlarmSettings(): Promise<TGetAlarmSettingsResp> {
-    const { data } = await axiosInstance.get<TGetAlarmSettingsResp>('/api/v1/alarms/settings');
+export async function getAlarmSettings(config?: AxiosRequestConfig): Promise<TGetAlarmSettingsResp> {
+    const { data } = await axiosInstance.get<TGetAlarmSettingsResp>('/api/v1/alarms/settings', { ...config });
     return data;
 }
 
 // 업데이트
-export async function patchAlarmSettings(payload: TAlarmSettings): Promise<TPatchAlarmSettingsResp> {
-    const { data } = await axiosInstance.patch<TPatchAlarmSettingsResp>('/api/v1/alarms/settings', payload);
+export async function patchAlarmSettings(payload: TAlarmSettings, config?: AxiosRequestConfig): Promise<TPatchAlarmSettingsResp> {
+    const { data } = await axiosInstance.patch<TPatchAlarmSettingsResp>('/api/v1/alarms/settings', payload, { ...config });
     return data;
 }

src/hooks/settingAlarm/useAlarms.ts (3)

12-16: React Query의 AbortSignal을 API로 전달해 네트워크 낭비를 줄여주세요

useCoreQuery의 queryFn에 전달되는 { signal }을 axios에 넘기면 탭 전환 등에서 불필요한 요청을 취소할 수 있습니다. API 레이어가 config?: AxiosRequestConfig를 받도록 확장한 뒤, 아래처럼 연결해 주세요.

-import { getAlarmSettings, patchAlarmSettings } from '@/api/settingAlarm/alarm';
+import { getAlarmSettings, patchAlarmSettings } from '@/api/settingAlarm/alarm';
+import { alarmSettingsKeys } from '@/api/settingAlarm/alarm.keys';
 
 // 조회
 export function useGetAlarmSettings() {
-    return useCoreQuery<TGetAlarmSettingsResp, TAlarmSettings>(['alarmSettings'], getAlarmSettings, {
+    return useCoreQuery<TGetAlarmSettingsResp, TAlarmSettings>(alarmSettingsKeys.all, ({ signal }) => getAlarmSettings({ signal }), {
         select: (resp) => resp.result,
         refetchOnWindowFocus: false,
     });
 }

참고: alarmSettingsKeys는 아래 코멘트의 추가 파일을 참고해 주세요.

---

21-25: invalidateQueries 대신 setQueryData로 즉시 캐시 반영(UX 개선)

PATCH 응답에 최종 설정값이 포함되므로, 무효화→재요청 대신 캐시를 직접 갱신하면 UI가 즉시 반영됩니다.

 export function usePatchAlarmSettings() {
     const qc = useQueryClient();
     return useCoreMutation<TPatchAlarmSettingsResp, TAlarmSettings>(patchAlarmSettings, {
-        onSuccess: () => {
-            qc.invalidateQueries({ queryKey: ['alarmSettings'] });
-        },
+        onSuccess: (resp) => {
+            qc.setQueryData<TAlarmSettings>(alarmSettingsKeys.all, resp.result);
+        },
     });
 }

---

10-10: 쿼리 키 하드코딩 대신 키 상수/키 빌더 사용 권장

['alarmSettings'] 리터럴 대신 키 상수를 사용하면 오타/중복을 줄일 수 있습니다. 위 수정안에서 사용한 alarmSettingsKeys 파일을 신설해 주세요.

추가 파일(신규):

// src/api/settingAlarm/alarm.keys.ts
export const alarmSettingsKeys = {
    all: ['alarmSettings'] as const,
};

src/pages/notice/NoticeDetail.tsx (1)

16-16: 주석이 실제 흐름과 다릅니다. 표현만 정정해 주세요

훅 호출 이전이 아니라, 유효성 검사 결과에 따라 렌더링을 가드하는 흐름입니다. 아래처럼 주석만 바로잡으면 혼동이 줄어듭니다. 코드 로직은 그대로 두고, enabled로 쿼리를 제어하는 현재 방식이 안전합니다.

-    // Id 유효성 검사 -> 훅 호출
+    // Id 유효성 검사 -> 렌더링 가드 (데이터 쿼리는 훅 내부 enabled 옵션으로 제어)

src/types/faq/faq.ts (1)

3-7: FAQ 타입 필드명이 컴포넌트 사용과 일치하지 않습니다

타입 정의의 title과 content 필드가 Question.tsx에서 사용하는 question과 answer 필드와 일치하지 않아 런타임 오류가 발생할 수 있습니다.

API 응답과 컴포넌트 인터페이스를 일치시켜야 합니다:

export type TFaqItem = {
    faqId: number;
-    title: string;
-    content: string;
+    question: string;
+    answer: string;
};

src/api/faq/faq.ts (1)

5-28: FAQ API 에러 처리 로직 추가 필요

현재 axios 에러가 그대로 전파되는데, FAQ 관련 특별한 에러 처리가 필요한지 검토해보세요.

공통 에러 처리를 추가하면 더 나은 사용자 경험을 제공할 수 있습니다:

+import { AxiosError } from 'axios';
+
// FAQ 목록 조회
export const getFaqs = async (params: { category: TFaqCategory; page: number; size: number }) => {
+    try {
        const { data } = await axiosInstance.get<TFetchFaqsResponse>('/api/v1/faqs', {
            params: {
                faqCategory: params.category,
                page: params.page,
                size: params.size,
            },
        });
        return data;
+    } catch (error) {
+        const axiosError = error as AxiosError;
+        console.error('FAQ 조회 실패:', axiosError.response?.data);
+        throw error;
+    }
};

src/pages/setting/DeleteConfirmPage.tsx (2)

43-47: 에러 타입 처리 개선 필요

any 타입 사용을 피하고 적절한 에러 타입을 사용해야 합니다.

-        onError: (error) => {
-            const msg = error?.response?.data?.message || '회원 탈퇴에 실패했습니다.';
+        onError: (error: unknown) => {
+            const axiosError = error as AxiosError<{ message?: string }>;
+            const msg = axiosError?.response?.data?.message || '회원 탈퇴에 실패했습니다.';
             alert(msg);
         },

---

120-120: Tailwind CSS 표준 클래스 사용 필요

rounding-32는 Tailwind CSS 표준 클래스가 아닙니다.

-className={`px-8 py-3 rounding-32 font-body1 text-white
+className={`px-8 py-3 rounded-full font-body1 text-white

src/components/settingTab/InfoSetting.tsx (4)

29-33: 빈 문자열 처리 로직 개선 필요

현재 조건문이 truthy 체크라 빈 문자열('')일 때 상태가 갱신되지 않습니다.

 useEffect(() => {
-    if (apiNickname) {
+    if (apiNickname !== undefined && apiNickname !== null) {
         setNickname(apiNickname);
         setInitialNickname(apiNickname);
     }
 }, [apiNickname]);

---

98-98: pointer-events-none 제거 권장

readOnly 속성만으로도 입력 방지가 충분합니다.

-                className="pointer-events-none"

---

111-111: Tailwind CSS 표준 클래스 사용 필요

rounding-32는 표준 Tailwind 클래스가 아닙니다.

-className={`bg-primary-500 px-6 py-2 rounding-32 text-white font-body1 ${resetPending ? 'opacity-60 cursor-not-allowed' : ''}`}
+className={`bg-primary-500 px-6 py-2 rounded-full text-white font-body1 ${resetPending ? 'opacity-60 cursor-not-allowed' : ''}`}

---

14-14: 에러 처리 헬퍼 함수의 타입 개선 필요

any 타입 대신 명시적인 타입을 사용하세요.

+import type { AxiosError } from 'axios';
-const getApiErrorMessage = (err: any, fallback: string) => err?.response?.data?.message ?? (err?.response?.status === 401 ? '로그인이 필요합니다.' : fallback);
+const getApiErrorMessage = (err: unknown, fallback: string) => {
+    const axiosError = err as AxiosError<{ message?: string }>;
+    return axiosError?.response?.data?.message ?? 
+           (axiosError?.response?.status === 401 ? '로그인이 필요합니다.' : fallback);
+};

src/hooks/faq/useFaq.ts (1)

16-16: TypeScript 제네릭 문법 오류 수정 필요

typeof 연산자에는 타입 인수를 전달할 수 없습니다. 옵션 타입을 올바르게 추출해주세요.

-export function useGetFaqs(params: TFaqListParams, options?: Parameters<typeof useCoreQuery<TFetchFaqsResponse>>[2]) {
+export function useGetFaqs(params: TFaqListParams, options?: Parameters<typeof useCoreQuery>[2]) {

-export function useSearchFaqs(params: TFaqSearchParams, options?: Parameters<typeof useCoreQuery<TFetchFaqsResponse>>[2]) {
+export function useSearchFaqs(params: TFaqSearchParams, options?: Parameters<typeof useCoreQuery>[2]) {

Also applies to: 26-26

src/api/notice/notice.ts (2)

5-15: 기존 호출부 호환성을 위한 별칭 export 추가 권장

fetchNotices를 사용하는 기존 코드와의 호환성을 유지하기 위해 임시 별칭을 추가하는 것이 좋습니다.

파일 하단에 다음 코드를 추가해주세요:

// 하위 호환성을 위한 임시 export - 추후 제거 예정
export { getNotices as fetchNotices, getNoticeDetail as fetchNoticeDetail };

---

36-36: 파일 끝 개행 문자 누락

Prettier 포맷팅 규칙에 따라 파일 끝에 개행을 추가해주세요.

     return data;
-};
+};
+

src/pages/notice/Notice.tsx (1)

61-74: 버튼 타입 명시 완료

type="button" 속성이 올바르게 추가되어 있어 의도하지 않은 폼 제출을 방지합니다.

src/components/common/PasswordEdit.tsx (1)

79-85: 취소 버튼 타입 명시 필요

폼 제출 방지를 위해 type="button" 추가가 필요합니다.

 <button
+    type="button"
     onClick={handleCancel}
     className="px-4 py-1.5 rounding-16 bg-default-gray-400 text-default-gray-700 font-body1 disabled:opacity-60"
     disabled={isPending}
 >

src/hooks/notice/useNotice.ts (1)

28-28: TypeScript 빌드 오류: typeof 연산자에 제네릭 타입 적용 불가

TypeScript에서는 typeof 연산자에 제네릭을 적용할 수 없습니다. 빌드 오류를 해결하려면 제네릭을 제거해야 합니다.

다음과 같이 수정하세요:

-    const useGetNotices = (params: TListParams, options?: Parameters<typeof useCoreQuery<TFetchNoticesResponse>>[2]) => {
+    const useGetNotices = (params: TListParams, options?: Parameters<typeof useCoreQuery>[2]) => {
...
-    const useGetNoticeDetail = (id: number, options?: Parameters<typeof useCoreQuery<TFetchNoticeDetailResponse>>[2]) =>
+    const useGetNoticeDetail = (id: number, options?: Parameters<typeof useCoreQuery>[2]) =>
...
-    const useSearchNotices = (params: TSearchParams, options?: Parameters<typeof useCoreQuery<TFetchNoticesResponse>>[2]) => {
+    const useSearchNotices = (params: TSearchParams, options?: Parameters<typeof useCoreQuery>[2]) => {

Also applies to: 45-45, 52-52

src/pages/question/Question.tsx (4)

37-37: 불필요한 useMemo 의존성 제거

size는 상수값(10)이므로 의존성 배열에 포함할 필요가 없습니다.

         }),
-        [active.value, currentPage],
+        [active.value, currentPage],
     );

// 마찬가지로 paramsSearch에서도:
         }),
-        [searchValue, active.value, currentPage],
+        [searchValue, active.value, currentPage],
     );

Also applies to: 48-48

---

52-60: placeholderData 중복 설정 제거

useFaq 훅 내부에서 이미 placeholderData: keepPreviousData를 설정하고 있으므로 중복 제거가 필요합니다.

-import { keepPreviousData } from '@tanstack/react-query';
...
     const listQuery = useGetFaqs(paramsList, {
         enabled: !hasKeyword,
-        placeholderData: keepPreviousData,
     });

     const searchQuery = useSearchFaqs(paramsSearch, {
         enabled: hasKeyword,
-        placeholderData: keepPreviousData,
     });

---

82-82: 빈 함수 핸들러 제거 권장

onSearchClick prop에 빈 함수를 전달하고 있습니다. 필수가 아니라면 제거하는 것이 좋습니다.

-                onSearchClick={() => {}}

또는 검색 시 페이지와 상태를 초기화하는 로직을 구현하세요:

-                onSearchClick={() => {}}
+                onSearchClick={() => {
+                    setCurrentPage(1);
+                    setOpenedIndex(null);
+                }}

---

90-103: 버튼에 type 속성 추가

버튼 요소에 명시적인 type 속성이 없어 form 내부에서 의도치 않은 submit이 발생할 수 있습니다.

                 {CATEGORY_MAP.map((c) => (
                     <button
+                        type="button"
                         key={c.value}
                         onClick={() => {

src/hooks/auth/useAccount.ts (1)

51-51: resetPreferences 직접 전달 가능

resetPreferences 함수가 파라미터를 받지 않으므로 화살표 함수 래퍼 없이 직접 전달할 수 있습니다.

-        return useCoreMutation<TResetPreferencesResponse, void>(resetPreferences, options);
+        return useCoreMutation<TResetPreferencesResponse, void>(resetPreferences, options);

현재 코드도 동작하지만, 불필요한 래퍼를 제거하면 더 간결해집니다.

src/types/auth/account.ts (1)

28-39: 도메인 값을 유니온 타입으로 제한 권장

타입 안정성 향상을 위해 백엔드 스키마에 맞춰 문자열 리터럴 유니온으로 제한하는 것이 좋습니다.

예시:

export type TMemberInfo = {
    id: number;
    email: string;
    username: string;
    userRank: 'BRONZE' | 'SILVER' | 'GOLD' | 'PLATINUM';
    phoneNumber: string;
    isAuthPayment: boolean;
    gender: 'MALE' | 'FEMALE' | 'UNKNOWN';
    birth: string;
    role: 'USER' | 'ADMIN';
    point: number;
};